Skip to content

Comments

Rename tag punctuation scope#7

Merged
vrugtehagel merged 1 commit intomainfrom
rename-tag-punctuation-scope
Jun 17, 2025
Merged

Rename tag punctuation scope#7
vrugtehagel merged 1 commit intomainfrom
rename-tag-punctuation-scope

Conversation

@vrugtehagel
Copy link
Collaborator

The HTML syntax definition relies on the punctuation.definition.tag.* scopes to provide automatic completions (e.g. when typing <p>< then inputting / completes to <p></p>). This clashes with Vento, causing HTML to complete to </{>. This renames the scope, using Liquid's naming instead; "embedded" instead of "tag".

The HTML syntax definition relies on the `punctuation.definition.tag.*`
scopes to provide automatic completions (e.g. when typing `<p><` then
inputting `/` completes to `<p></p>`). This clashes with Vento, causing
HTML to complete to `</{>`. This renames the scope, using Liquid's
naming instead; "embedded" instead of "tag".
@vrugtehagel vrugtehagel merged commit 8ac84e4 into main Jun 17, 2025
1 check passed
@vrugtehagel vrugtehagel deleted the rename-tag-punctuation-scope branch June 17, 2025 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant